Skip to content

Fix/eng 11829 creating new account - #11888

Closed
Vlad0n20 wants to merge 10 commits into
CenterForOpenScience:feature/gdpr-delete-orcid-rewritefrom
Vlad0n20:fix/ENG-11829-creating-new-account
Closed

Fix/eng 11829 creating new account#11888
Vlad0n20 wants to merge 10 commits into
CenterForOpenScience:feature/gdpr-delete-orcid-rewritefrom
Vlad0n20:fix/ENG-11829-creating-new-account

Conversation

@Vlad0n20

Copy link
Copy Markdown
Contributor

Ticket

Purpose

Changes

Side Effects

QE Notes

CE Notes

Documentation

@Vlad0n20
Vlad0n20 force-pushed the fix/ENG-11829-creating-new-account branch from a34fd0e to 3493932 Compare September 1, 2026 15:09
Comment thread osf/models/user.py
Comment on lines +2083 to +2098
with transaction.atomic():
self._validate_and_remove_resource_for_gdpr_delete(
self.nodes.all(),
hard_delete=False
)
self._validate_and_remove_resource_for_gdpr_delete(
self.preprints.all(),
hard_delete=False
)
self._validate_and_remove_resource_for_gdpr_delete(
self.draft_registrations.all(),
hard_delete=True
)

# Finally delete the user's info.
self._clear_identifying_information()
# Finally delete the user's info.
self._clear_identifying_information()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part is lost in the replacement PR #11899, I will add it back.

@cslzchen

cslzchen commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

PR replaced by #11899 due to squash merge/conflicts.

@cslzchen cslzchen closed this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants